Skip to content

Conversation

@xingzhang-suse
Copy link
Collaborator

The commits are separated for different fixed location. It will be easier for reviewing.
The final layout details could be different from v5.4.8.
What do should be check is the layout broken or alignment issue.

…lasses (Replaced mr- into me-, removed mr-lg, mr-sm)
…lasses (Replaced ml- into ms-, removed ml-lg, ml-md, ml-sm)
…lasses (Fixed styling issues on the top navbar)
…lasses (Fixed Card component, font and login page)
…lasses (Replaced pr- into pe-, pl- into ps-)
…lasses (Fixed Assets pages excluding Registries page)
@xingzhang-suse xingzhang-suse requested a review from a team as a code owner January 6, 2026 17:00
@xingzhang-suse xingzhang-suse requested review from lsongsuse and removed request for a team January 6, 2026 17:00
@rushk014
Copy link
Contributor

LGTM. There is a lot of alignment / spacing issues we will need to fine tune still. Apart from that there are a few functionality bugs:

  1. ngx-carousel css is broken - since it is deprecated we should migrate to ngu-carousel
ngx-carousel
  1. Close button is broken on Network Activity page
Screenshot 2026-01-13 at 10 39 21 AM
  1. angular-notifier is broken in Angular 14+ resulting in broken notifications across the whole application. There is a maintained fork here we can try to migrate to
Screenshot 2026-01-13 at 10 40 43 AM
  1. Several pages do not render translated i18n strings. I suspect this is a dependency injection issue but we should take a closer look.
Screenshot 2026-01-13 at 10 42 13 AM Screenshot 2026-01-13 at 10 45 54 AM Screenshot 2026-01-13 at 10 46 08 AM

@rushk014 rushk014 merged commit 309c5fb into neuvector:angular20 Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants